Skip to content

Conversation

@matthew-kapp
Copy link
Contributor

The examples have been updated in such a way that the exact same model is created, but the syntax matches that used by MTK v9, where @mtkmodel and other macros are used.

  1. electrical.jl is updated with electrical.svg generated. For some reason, the capacitor is skew though.
  2. mechanical.jl is updated with mechanical.svg generated.
  3. hydraulic.jl was not updated, as components within @mtkmodel expect (; name) as the firs argument. Tube in standard library is not set up this way.

@matthew-kapp
Copy link
Contributor Author

@bradcarman Just following up here?

@bradcarman
Copy link
Owner

Yeah, I think this looks good. Can you just remove Revise from the Project.toml? I don't think it needs to be there.

@matthew-kapp
Copy link
Contributor Author

@bradcarman Done

@matthew-kapp
Copy link
Contributor Author

@bradcarman Just following up here?

@bradcarman
Copy link
Owner

Hi @matthew-kapp this looks good, I'll approve. Thanks for your contribution!

@bradcarman bradcarman self-requested a review February 26, 2025 14:02
Copy link
Owner

@bradcarman bradcarman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All looks good

@matthew-kapp
Copy link
Contributor Author

@bradcarman sorry, I'm still learning git approvals - is there still a final step to merge merge request?

@bradcarman bradcarman merged commit 3eda1bf into bradcarman:master Feb 27, 2025
1 check passed
@bradcarman
Copy link
Owner

Sorry, I was just waiting for tests to pass, all merged now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants